Codecov Report

❌ Patch coverage is 64.28571% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.24%. Comparing base (e208db7) to head (088f424).

@@            Coverage Diff             @@
##           master     #136      +/-   ##
==========================================
- Coverage   53.25%   53.24%   -0.02%     
==========================================
  Files          23       23              
  Lines        7776     7806      +30     
  Branches     1797     1804       +7     
==========================================
+ Hits         4141     4156      +15     
- Misses       2607     2619      +12     
- Partials     1028     1031       +3     

Why would you want that, what is the usecase?
There is already a dl mechanism in clixon you can use to dynamically add symbols for callbacks, helptexts, etc.
If there is something that you want to extend, please do that on a clixon api level instead, I dont want to add a dl extension layer in cligen as well.
If this is related to the expand-dbvar discussion in matrix, I would rather add an indirection layer (or callback mechanism) in expand-dbvar instead in clixon.

@olofhagsand
I want the user in the client to receive a list of current mac addresses when clicked, which he does not fill out in advance.

sysadmin@eve-img:/> set system interface management mgmt0 mac-address
50:02:00:88:00:00 Interface is enp0s2
50:02:00:88:00:03 Interface is enp0s5
50:02:00:88:00:04 Interface is enp0s6

an important note is that where this is inserted is declared in yang

in the cli, it's done like this
set @datamodel, cli_auto_set();
I can't use dbvar.

There is already a dl mechanism in clixon you can use to dynamically add symbols for callbacks, helptexts, etc.
If there is something that you want to extend, please do that on a clixon api level instead, I dont want to add a dl extension layer in cligen as well.

@olofhagsand
yes it is, but it only works for runtime commands. to do this, I have to describe them in the .cli, but if I have
set @datamodel, cli_auto_set(); unfortunately, I can't do that.

and what I need is inside the generated cli inside the set.

One could extend he syntax to make the expand function configurable,
two ideas could be:

1. set @datamodel, cli_auto_set(), myexpand();
2. or to make a yang cli extension where you tag the YANG with another expand function

2. or to make a yang cli extension where you tag the YANG with another expand function

this will only work when the client is running, and if something has changed in runtime, I will not be able to update it.

1. set @datamodel, cli_auto_set(), myexpand();

is this, as I understand it, not implemented? or is it already there? if there is a place to read about it?

No it is not implemented.

@olofhagsand Maybe we can add this solution for now, and fix it as soon as another one appears? I really need this functionality now. And to do this, set @datamodel, cli_auto_set(), myexpand(); I don't know how yet.

You can add this on your local repo if you like. I will not add an intermediate solution in upstream.
BTW, a third option is to add a callback in expand_dbvar itself, which is probably the easiest to implement. I.e., expand_dbvar would call that callback function if it exists, if not it would return the default expand function.
This is something one could implement relatively easy.
In other words, add another indirection in the autocli code (lib/src/clixon_autocli_generate.c) , ie instead of:

/* variable expand function */
#define GENERATE_EXPAND_XMLDB "expand_dbvar"

As a first step, instead of it being fixed in C, it could be added to include/clixon_custom.h or in configure.ac.
As a second step, it could be a callback function for the CLI which you register in clixon_plugin_init().
This would be straightforward.